Skip to content

MCP-744 Use shared Gitar Renovate safety rule - #733

Merged
nquinquenel merged 1 commit into
masterfrom
task/nq/deduplicate-gitar-renovate-rules
Nov 12, 2026
Merged

nquinquenel merged 1 commit into
masterfrom
task/nq/deduplicate-gitar-renovate-rules

Conversation

@nquinquenel

Copy link
Copy Markdown
Member

Use the shared Renovate safety automation from gitar-config#6 by removing the local .gitar/rules/renovate-safety.md override and the label-only renovate-major-minor package rule. The shared rule identifies eligible major/minor upgrades without labels.

Retains Gradle lockfile regeneration and the existing squad review focus.

Validation:

  • Parsed the Renovate JSON and confirmed every remaining setting is unchanged. Checked for stale rule/label references and ran git diff --check.
  • DEEP Vortex passed for .github/renovate.json with no issues.

Application builds are unnecessary for this configuration-only change. Static checks do not prove live shared-rule inheritance; an eligible Renovate PR is needed to verify the automation end to end.

@nquinquenel
nquinquenel requested a review from a team as a code owner November 12, 2026 10:32
@nquinquenel
nquinquenel requested a review from Krosovok November 12, 2026 10:32
@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod Bot changed the title Use shared Gitar Renovate safety rule MCP-744 Use shared Gitar Renovate safety rule Nov 12, 2026
@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

MCP-744

@gitar-bot

gitar-bot Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Code Review ✅ Approved 🟡 Medium risk

Removing the local Renovate safety rule relies on shared inheritance; if it is not applied, major/minor upgrades may lack intended safeguards.

Removes the local Renovate safety rule override and the label-only renovate-major-minor package rule in favor of the shared rule from gitar-config#6, with Gradle lockfile regeneration and the squad review focus retained. No issues found.

Verification Rules 2/2 met PRs 1 inspected

Functional validation No results

Rules 2 passed 2 not applicable

✅ Internal artifact registries
✅ Repository license compliance

Broader impact PRs 1 inspected
Cross-PR 1 PR inspected 0 open impacts

PR 2061: SLI-3060 Use shared Gitar Renovate safety rule Inspected

Auto-approval Not enabled

Set up

For more verification details, comment gitar display:verbose.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing compact verification results.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqubecloud

Copy link
Copy Markdown

@nquinquenel
nquinquenel merged commit 75f0ec4 into master Nov 12, 2026
17 checks passed
@nquinquenel
nquinquenel deleted the task/nq/deduplicate-gitar-renovate-rules branch November 12, 2026 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants